Upgrade spdy to 4.x.x and support Node v6, v8, v10
#373
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ient. Fix issue with stream deduping when finding intenral remote port in the spdy internals.
Updates to the WS cause it to return a 400 instead of a 404 when the base path does not match.
…ribe's cb to fire a second time. Wrap cb in a once pattern. PeerSocket guard against this.ws not being initialized when closing the connection
Our tests are scheduling async operations and calling done. We need to clean this up but with the old version of mocha it was exiting the test suite after all tests were done. This enforces the old behavior.
This ensures zetta hubs running pre spdy-upgrade will be compatible at a network level. To support `h2` we need to negotiate transport and ALPN cannot help us.
|
Opened a new PR since i lost access to the repo after leaving Google. #379 I also published version [email protected] that incorporates all these changes. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Well it's been about two years in making but this PR allows Zetta to run on the latest version of Node.js with the
spdyandwsmodule updated to their newest versions. Sorry for all the delay, folks.This release adds support for the versions below and drops supports for node <= v4. v0.10, v0.12, etc.. Sorry for anyone who uses it but those versions are long retired. Fixes #336, #333 and will make progress with #337. Once merged we can move much fast to modernize the codebase.
Peering will work between old versions of zetta and this version and vise-versa.
Node.js Release Support (Node Releases)
v11 is going to require some work on the node-spdy front. see: spdy-http2/node-spdy#350. Or by then we can move off
node-spdyin favor of the http2 core module.Major Changes to Spdy
pingevent from the transports Connection. spdy-http2/node-spdy#353 open to add support for receiving pings offically until then https://github.com/zettajs/node-spdy supports it.There were also changes in
wsthat broke getting the request url from the ws object. Instead we need to pass the originalrequestto thePeerSocket.TODO
argoandtitanto a version with no security vulnerabilities.